[feat] 동적 inflate를 위한 chip 관련 구현, BreadTypeList Dto 형태 수정 및 관련 뷰 수정 #227

Dan2dani · 2024-02-17T05:38:47Z

Related issue 🛠

closed [mod] BreadType, NutrientType DTO 수정 및 로직 수정(마이페이지, 내가 쓴 리뷰, 저장 목록) #224

Work Description ✏️

공통으로 쓰이는 BreadTypeIdDto, NutrientTypeIdDto 분리
BreadType, NutrientType Dto 형태 수정에 따른 관련 Dto, Entity 형식 수정
BreadType, NutrientType Dto 매핑할 때, 관리를 용이하게 하기위해서, id형태로 떨어지는 값을 -> enum 형태로 매핑해서 사용했습니다.
(관리 + 랜더링 시점에서 리스트를 만들어내는게 비효율적이라고 생각)
ChipFactory에서 다양한 디자인의 Chip을 만들 수 있도록 Chip Factory 수정 (관련해서 차후 논의 필요할 듯)
Chip inflate, ChipGroup addView를 위한 각종 확장함수 구현
Dto 수정에 따른 BreadTypeChips 랜더링 방식 수정(가시성 조절 -> chip 동적 추가 형태로)

Screenshot 📸

건빵집 상세페이지

리뷰 작성하기

나 이거 사진이 안올라감

Uncompleted Tasks 😅

건빵집 리스트 Dto 수정 (-> 차후에, 페이지네이션으로 수정하면서 수정 예정)

To Reviewers 📢

차후에 논의

List로 전부 전환 후 기존 엔티티 삭제 예정

…hips 호출하고, 람다식 작성

…Type-chip

jihyunniiii

수고하셨숩니당 ~

jihyunniiii · 2024-02-17T09:55:53Z

app/src/main/java/com/sopt/geonppang/data/model/response/BreadTypeIdDto.kt

따로 만들어서 관리하니까 확실히 코드가 깔끔해지는군요 짱짱쓰

jihyunniiii · 2024-02-17T09:57:43Z

app/src/main/java/com/sopt/geonppang/data/model/response/ResponseSettingFilter.kt

+        breadTypeList = data.breadTypeList.mapNotNull { breadType ->
+            BreadFilterType.values().find {
+                it.id == breadType.breadTypeId
+            }
+        },


오 훨씬 좋네용

enum으로 바꾸는거 굳이죵 ~~

jihyunniiii · 2024-02-17T09:58:29Z

app/src/main/java/com/sopt/geonppang/domain/model/Bakery.kt

@@ -12,3 +15,17 @@ data class Bakery(
    val bakeryPicture: String,
    val breadType: BreadType,
 )
+
+// TODO: 차후 네이밍 수정
+data class BakeryInformation(


이 네이밍도 괜찮을 듯요? ㅋ.ㅋ

그런가염? 뭐가 나을까요

BakeryIdentity 추천해봅니당

jihyunniiii · 2024-02-17T09:59:12Z

app/src/main/java/com/sopt/geonppang/presentation/bakeryList/BakeryListFragment.kt

@@ -75,7 +75,7 @@ class BakeryListFragment :
        viewModel.bakeryListState.flowWithLifecycle(lifecycle).onEach {
            when (it) {
                is UiState.Success -> {
-                    bakeryAdapter.setBakeryList(it.data.toMutableList())
+//                    bakeryAdapter.setBakeryList(it.data.toMutableList())


여기 왜 주석 처리 되어있나염?????

여기 페이징 전환할 대 하려구요 ㅋㅎ

jihyunniiii · 2024-02-17T10:01:04Z

app/src/main/java/com/sopt/geonppang/presentation/myPage/MyPageFragment.kt

+        // 생명 주기 상태가 최소 CREATED 일 때만 수집하도록 제한
+        viewModel.profileInfo.flowWithLifecycle(
+            viewLifecycleOwner.lifecycle,
+            Lifecycle.State.CREATED


와 핑글 때 멘토님 피드백이 이런 의미였군요 감솨합니다 ~

jihyunniiii · 2024-02-17T10:02:18Z

app/src/main/java/com/sopt/geonppang/presentation/myPage/MyReviewAdapter.kt

+
+            // TODO: dana 다른 방식이 있는지 고민, 매 바인딩마다 removeAllViews 해야하는가 ?
+            with(binding.cgBakeryBreadTypes) {
+                this.removeAllViews()


요거 매 바인딩마다 안 하면 칩이 안 사라지는 문제가 발생하게 되나요? 재사용 때문엥?

칩이 동적으로 바인딩되는거라서 원래 있던거에 추가로 생김요 ㅜㅜ

무슨 방법 없을까여 근데 이게 제일 깔끔한 거 같기도,,,

jihyunniiii · 2024-02-17T10:02:54Z

app/src/main/java/com/sopt/geonppang/util/ChipFactory.kt

이게 이미 있었다니 충격 ㄷ.ㄷ

jooyyoo

수고했어얌🩵

jooyyoo · 2024-02-17T15:14:17Z

app/src/main/java/com/sopt/geonppang/domain/model/Bakery.kt

@@ -12,3 +15,17 @@ data class Bakery(
    val bakeryPicture: String,
    val breadType: BreadType,
 )
+
+// TODO: 차후 네이밍 수정
+data class BakeryInformation(


BakeryIdentity 추천해봅니당

jooyyoo · 2024-02-17T15:17:11Z

app/src/main/java/com/sopt/geonppang/util/extension/ChipExt.kt

마술사 같다...

Dan2dani added 15 commits February 15, 2024 22:46

[mod] #224 여러 디자인에 대한 Chip Factory를 만들 수 있도록 수정

152b9e9

[mod] #224 sdk 올리기 및 각종 라이브러리 버전 업데이트

aac156f

[feat] #224 BreadTypeIdDto, NutrientTypeIdDto 같이 공통으로 쓰이는 Dto 분리

f976b4e

[feat] #224 breadType에 대해 List 형태로 가지고 있는 엔티티 생성

ecffe6e

List로 전부 전환 후 기존 엔티티 삭제 예정

[feat] #224 breadType model 형태 수정에 따른 관련 model, entity 형태 수정

86095df

[feat] #224 breadType chip 디자인

2bdfcc9

[feat] #224 chip inflate에 필요한 각종 확장함수 구현

3f280a7

[mod] #224 breadType이 포함된 뷰 디자인 수정(chip 삭제)

a504b7c

[feat] #224 list item마다 chipgroup이 inflate 될 수 읷도록, onBind 함수에서 initC…

e546acf

…hips 호출하고, 람다식 작성

[feat] #224 마이페이지에 나의 BreadType chip 동적 생성하기

560661d

[feat] #224 나의 저장목록 BreadTypeChips 동적 생성

1c3468c

[feat] #224 나의 리뷰 모아보기 BradTypeChips 동적 생성

eb15885

[feat] #224 나의 리뷰 상세보기, 빵집 BreadTypeChips 동적 생성

0310713

[feat] #224 검색 결과 리스트 breadTypeChips 동적 생성

47b65cd

[feat] #224 리뷰 작성, 빵집 상세뷰 BreadTypsChips 동적 생성

78c60a0

Dan2dani added 🐰 다은 🐰 💫 feat 💫 labels Feb 17, 2024

Dan2dani added this to the [릴리즈] 4차 스프린트 milestone Feb 17, 2024

Dan2dani requested review from jooyyoo and jihyunniiii February 17, 2024 05:38

Dan2dani self-assigned this Feb 17, 2024

Dan2dani added 3 commits February 17, 2024 14:54

[chroe] #224 ktlint 적용

7a2b76c

[chroe] #224 ktlint 적용

60ad988

Merge remote-tracking branch 'origin/mod-bradType-chip' into mod-brad…

9aff4bc

…Type-chip

jihyunniiii approved these changes Feb 17, 2024

View reviewed changes

jooyyoo approved these changes Feb 17, 2024

View reviewed changes

Dan2dani merged commit 0f3a1ac into develop Feb 18, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] 동적 inflate를 위한 chip 관련 구현, BreadTypeList Dto 형태 수정 및 관련 뷰 수정 #227

[feat] 동적 inflate를 위한 chip 관련 구현, BreadTypeList Dto 형태 수정 및 관련 뷰 수정 #227

Dan2dani commented Feb 17, 2024 •

edited

Loading

jihyunniiii left a comment

jihyunniiii Feb 17, 2024

jihyunniiii Feb 17, 2024

Dan2dani Feb 17, 2024

jihyunniiii Feb 17, 2024

Dan2dani Feb 17, 2024

jooyyoo Feb 17, 2024

jihyunniiii Feb 17, 2024

Dan2dani Feb 17, 2024

jihyunniiii Feb 17, 2024

jihyunniiii Feb 17, 2024

Dan2dani Feb 17, 2024

Dan2dani Feb 17, 2024

jihyunniiii Feb 17, 2024

jooyyoo left a comment •

edited

Loading

jooyyoo Feb 17, 2024

jooyyoo Feb 17, 2024

[feat] 동적 inflate를 위한 chip 관련 구현, BreadTypeList Dto 형태 수정 및 관련 뷰 수정 #227

[feat] 동적 inflate를 위한 chip 관련 구현, BreadTypeList Dto 형태 수정 및 관련 뷰 수정 #227

Conversation

Dan2dani commented Feb 17, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

jihyunniiii left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jooyyoo left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Dan2dani commented Feb 17, 2024 •

edited

Loading

jooyyoo left a comment •

edited

Loading